Skip to content

Conversation

akh7177
Copy link
Contributor

@akh7177 akh7177 commented Oct 4, 2025

Replaced the outdated Error Msg and Claimed Username

Copy link
Contributor

github-actions bot commented Oct 4, 2025

Automatic validation of changes

Target F+ Check F- Check
HackerEarth ❌   Fail ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@akh7177
Copy link
Contributor Author

akh7177 commented Oct 4, 2025

Hello @ppfeister ,

Even though F+ are not getting triggered when I use random, permitted usernames, automatic validation seems to be failing. Could you pls have a look into it?

Thank you!

@akh7177 akh7177 changed the title Remediate False Positives for HackerEarth fix(sites): Remediate False Positives for HackerEarth Oct 4, 2025
Copy link
Contributor

github-actions bot commented Oct 4, 2025

Automatic validation of changes

Target F+ Check F- Check
Roblox ❌   Fail ✔️   Pass
threads ❌   Fail ✔️   Pass
CyberDefenders ✔️   Pass ✔️   Pass
HackerEarth ❌   Fail ✔️   Pass
SlideShare ❌   Fail ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@ppfeister
Copy link
Member

ppfeister commented Oct 4, 2025

So this one checks out locally.... Unsure as to why it's failing on the runner.

Will be working on improving the tests so it doesn't get automatically excluded. Will merge this shortly and use this as one of my test cases.

@ppfeister ppfeister self-assigned this Oct 4, 2025
@akh7177
Copy link
Contributor Author

akh7177 commented Oct 5, 2025

So this one checks out locally.... Unsure as to why it's failing on the runner.

Will be working on improving the tests so it doesn't get automatically excluded. Will merge this shortly and use this as one of my test cases.

Sure, thank you

@ppfeister
Copy link
Member

Added the label in case it gets lost in the meantime --- should give credit as of today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants